Skip to content

Prevent duplicate CI runs in test_robusta workflow #1993

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
Sheeproid merged 1 commit into from
Jan 14, 2026
Merged

Prevent duplicate CI runs in test_robusta workflow #1993

Sheeproid merged 1 commit into from
Jan 14, 2026
+5 −1

Conversation

@Sheeproid
Copy link
Contributor

@Sheeproid Sheeproid commented Jan 14, 2026 •
edited
Loading

Configure workflow to only run on pushes to master branch and all pull requests. This prevents duplicate runs when pushing to a PR branch (both push and pull_request events).

Before - 2 test of each kind that were the same

Screenshot 2026-01-14 at 16 45 03

After - only 1 test of each kind!

Screenshot 2026-01-14 at 17 10 30

@claude
Prevent duplicate CI runs in test_robusta workflow
ab153bd 
Configure workflow to only run on pushes to master branch and all pull requests.
This prevents duplicate runs when pushing to a PR branch (both push and pull_request events).
@coderabbitai
Copy link

coderabbitai bot commented Jan 14, 2026 •
edited
Loading

Walkthrough

Updated the GitHub Actions workflow trigger configuration in .github/workflows/test_robusta.yaml from a compact array format to an explicit mapping that adds a branch filter, restricting push events to the master branch while retaining pull_request and workflow_dispatch triggers.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/test_robusta.yaml		 Restructured workflow trigger from array format [push, pull_request, workflow_dispatch] to explicit mapping with branch filtering on push events (master branch only)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: preventing duplicate CI runs by modifying workflow triggers to run on specific branches.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description check ✅ Passed The pull request description clearly explains the purpose of the workflow change and demonstrates the fix with before/after screenshots showing the prevention of duplicate CI runs.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch claude/add-greeting-message-uPfAJ

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link

github-actions bot commented Jan 14, 2026 •
edited
Loading

✅ Docker image ready for 0fd748a (built in 3m 25s)

⚠️ Warning: does not support ARM (ARM images are built on release only - not on every PR)

Use this tag to pull the image for testing.

📋 Copy commands

⚠️ Temporary images are deleted after 30 days. Copy to a permanent registry before using them:

gcloud auth configure-docker us-central1-docker.pkg.dev
docker pull us-central1-docker.pkg.dev/robusta-development/temporary-builds/robusta-runner:0fd748a
docker tag us-central1-docker.pkg.dev/robusta-development/temporary-builds/robusta-runner:0fd748a me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:0fd748a
docker push me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:0fd748a

Patch Helm values in one line:

helm upgrade --install robusta robusta/robusta \
  --reuse-values \
  --set runner.image=me-west1-docker.pkg.dev/robusta-development/development/robusta-runner-dev:0fd748a

@Sheeproid Sheeproid requested a review from RoiGlinik January 14, 2026 15:11
@Sheeproid Sheeproid enabled auto-merge (squash) January 14, 2026 15:11
@Sheeproid Sheeproid merged commit 3e94bc1 into master Jan 14, 2026
7 checks passed
@Sheeproid Sheeproid deleted the claude/add-greeting-message-uPfAJ branch January 14, 2026 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@RoiGlinik RoiGlinik RoiGlinik approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@Sheeproid @RoiGlinik @claude